-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[PLA-1714] Add refresh csrf token #90
Conversation
PR Description updated to latest commit (65217fd) |
PR Review
Code feedback:
✨ Review tool usage guide:Overview: The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.
See the review usage page for a comprehensive guide on using this tool. |
PR Code Suggestions
✨ Improve tool usage guide:Overview:
See the improve usage page for a comprehensive guide on using this tool. |
Type
enhancement
Description
reloadCsrf
method inApiService
public and enhanced its implementation to fetch and parse CSRF token.Changes walkthrough
Settings.vue
Integrate CSRF Token Refresh in Settings Page
resources/js/components/pages/Settings.vue
ApiService
from~/api
.ApiService.reloadCsrf()
after clearing logininformation in the
deleteAccount
function.index.ts
Enhance CSRF Token Reloading Mechanism
resources/js/api/index.ts
reloadCsrf
method fromprotected
topublic
to allow accessfrom other components.
index.ts
Refresh CSRF Token on Login and Logout
resources/js/store/index.ts
ApiService.reloadCsrf()
after login and logoutprocesses to refresh CSRF token.